Skip to content

Conversation

vil02
Copy link
Member

@vil02 vil02 commented May 24, 2025

Description of Change

This PR removes some unused includes of the vector header. Such shell script was useful to identify the files needing an update:

#!/bin/bash

find . -type f \( -name "*.cpp" -o -name "*.hpp" \) | while read -r file; do
    if grep -q '<vector>' "$file" && ! grep -q 'vector<' "$file"; then
        echo "$file"
    fi
done

Checklist

  • Added description of change
  • PR title follows semantic commit guidelines
  • Search previous suggestions before making a new one, as yours may be a duplicate.
  • I acknowledge that all my contributions will be made under the project's license.

Notes:
Removed unused vector includes.

Copy link
Collaborator

@realstealthninja realstealthninja left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@realstealthninja realstealthninja merged commit 93a700c into TheAlgorithms:master May 25, 2025
7 checks passed
@vil02 vil02 deleted the remove_unused_vector_includes branch May 25, 2025 22:51
@vil02 vil02 mentioned this pull request May 26, 2025
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants